Merge branch 'inspector-list-model' into 'master'

Inspector list model support

See merge request GNOME/gtk!2176

inspector: Avoid a use of gtk_style_context_add_class

We have gtk_widget_add_css_class for this now.

inspector: proper list model support

Add a data tab for list models that allows exploring
the objects in the model.

Merge branch 'wip/on-the-surface-good-fences-can-make-bad-neighbors' into 'master'

x11: Handle window getting unmap while frame still pending

Closes #2902

See merge request GNOME/gtk!2168

inspector: Rename DataList -> TreeData

Rename the DataList object to TreeData, in preparation
for adding a ListData object for list models. While
we are touching it, modernize it a bit (drop the Private
struct, use a layout manager, etc).

x11: Avoid thawing surface until frame is drawn

Since commit 972134abe48a4c9c7b6ad41b0723f30f4e7ae16b a frame getting
drawn has three states (with the vendor nvidia driver at least):

1. drawn by gtk waiting on the GPU
2. drawn by GPU waiting on the compositor
3. drawn by compositor

Those three states are encoded in two flags: frame_pending and
frame_still_painting.

frame_pending means step 1 is done, but step 2 and 3 are still
in progress.  frame_still_painting means step 2 is still in progress.

After step 1 is finished the surface is frozen until step 3 is finished.

When the compositor notifies gtk it's done with step 3, with a
_NET_WM_FRAME_DRAWN client message, the toolkit thaws the surface to
allow the next frame to proceed.

The compositor sometimes sends gtk a _NET_WM_FRAME_DRAWN client message
between steps 1 and 2.  This message should be ignored because it's not
a reply to the current frame.

Unfortunately, gtk currently assumes if it gets a _NET_WM_FRAME_DRAWN
client message while waiting for step 2 that it's actually at step 3,
and proceeds to draw a new frame while the existing frame is still
pending, leading to a blown assertion.

This commit addresses the problem by ignoring _NET_WM_FRAME_DRAWN
client messages from the compositor unless actually expecting one.

Fixes: #2902

x11: Handle window getting unmapped while frame still pending

Since commit 972134abe48a4c9c7b6ad41b0723f30f4e7ae16b we now call
glClientWaitSync for the vendor nvidia driver, to know when a frame
is ready for the compositor to process.

If a surface is hidden while a frame is still being rendered by the GPU,
the surface will never produce the damage event the code relies on to
trigger the call to glClientWaitSync. This leaves the fence dangling,
and the next time the surface is shown, it will start a fresh frame
and blow an assertion since the fence from the last frame is still
hanging around.

This commit ensures a frame gets fully wrapped up before hiding a
surface.

inspector: Remove list model support from the property editor

This makes the inspector lock up when used with any production
size list model, and blocks access to properties of the model
itself. Instead, we'll make the model available as an object
and add a data tab for list model contents, like we already
do for tree models.

Merge branch 'issue-2904' into 'master'

Add nullable annotations for gtk_cclosure_expression_new()

Closes #2904

See merge request GNOME/gtk!2172

Add nullable annotations for gtk_cclosure_expression_new()

Fixes: #2904

gtklistitemfactory: Cosmetic documentation fixes

builderlistitemfactory: Documentation fixes

expression: Add to the docs

Add some more long-form explanation of what expressions
are about and where they are used.

Drop GtkFunctionsListItemFactory

It has been superseded by GtkSignalsListItemFactory.

Stop using GtkFunctionsListItemFactory

This was an early attempt at a factory, and has been
superseded by GtkSignalsListItemFactory. Port all users
the the newer one.

Merge branch 'wip/Jehan/GtkIMContextWayland-master' into 'master'

gtk: focus out the GtkIMContextWayland upon finalization.

See merge request GNOME/gtk!2170

gtk: focus out the GtkIMContextWayland upon finalization.

In particular, it will NULL-ified the current global context if this is
the finalized one, avoiding dangling invalid pointers.

Would have been a cherry-pick from branch gtk-3-24 of commit
b592ded80ad4825d115317ba6750a086da0434c0, but files moved.

Updated Spanish translation

Merge branch 'ci-style' into 'master'

ci: Create new origin for forks

See merge request GNOME/gtk!2164

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #1887

See merge request GNOME/gtk!2167

ci: Create new origin for forks

We don't need to create a new remote and fetch its master if we're
checking a merge request done on the upstream repository.

node-editor: Add a help window

Add a Help item to the gear menu that opens the
node-format.md file in a new window. This could
be improved if we could parse markdown and apply
tags, similar to how we can load pango markup.

node-editor: Cosmetics

Use a title style class for the labels.

node-editor: Add a gear menu

Add a gear menu with Inspector and About menu items.

node-editor: Document the node format

Add a markdown file with the documentation of the
node format.

Fixes: #1887

gtk-demo: Fix binary name in --version

We are gtk4-demo, not gtk3-demo.

bitset: Fix documentation syntax

bitset: Add more tests

Add some tests for rectangles, and for iters.

bitset: Correct preconditions in gtk_bitset_add_rectangle

We want to make sure that the rectangle fits in the grid.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #2743

See merge request GNOME/gtk!2161

ci: Drop one of the focus tests

One of the widget-factory focus tests is flaky in ci,
perhaps due to font changes causing size computations
to go slightly differently.

Drop this for now.

testsuite: Bump the per-test timeout to 60s

The bitset test has repeatedly run into the 30s timeout
in the ci, so give it some more time.

bitset: Fix the right-shift implementation

This was not doing the right thing at all.

This commit also adds tests for left- and
right-shift.

Stop setting GTK_IM_MODULE_FILE

GTK no longer reads this environment variable, so
setting it can have no benefit for uninstalled demos
anymore.

Add g_intern_static_string to valgrind suppressions

That function can also trigger a quark table reallocation.

widget: Avoid uninitialized access

Initialize all field in the GtkCrossingData struct
when synthesizing crossing events.

Fixes: #2743

gtk: Better help for GTK_DEBUG

Reuse the newly introduced gdk_parse_debug_var for
GTK_DEBUG.

gsk: Better help for GSK_DEBUG

Reuse the newly introduced gdk_parse_debug_var for
GSK_DEBUG.

gdk: Better help for GDK_DEBUG

Include docstrings and format the list of supported
values better.

Also, add the same warning we have for GTK_DEBUG when
the environment variable is ignored.

testsuite: Don't pass GDK_DEBUG for release builds

The debug env vars are ignored in release builds,
and may spew warnings about that fact that break
tests.

wayland: Respect GDK_DEBUG=default-settings

You can get this in other ways for Wayland (by
setting GSETTINGS_BACKEND=memory), but it is better
to be consistent across backends.

Drop GTK_DEBUG_SHORTCUTS

It is unused.

shortcuts: Use GTK_DEBUG_KEYBINDINGS

There was just a single message under GTK_DEBUG_SHORTCUTS.
Consolidate with GTK_DEBUG_KEYBINDINGS.

docs: Update environment sections

Fix links in markdown, and add details about
Wayland environment variables.

Merge branch 'print-editor' into 'master'

Install the print editor

See merge request GNOME/gtk!2165

stringlist: Cosmetic documentation fix

gtk: Add some missing symbols to the docs

sorter: Some doc corrections

The docs were referring to some non-existing enum value.
Fix things up to match the current code.

stack: Remove nonexisting api from headers

The homogeneous property was dropped a while ago.

gdk: Small documentation fixes

Make sure gdk_event_get_seat shows up.

stack: Cosmetic documentation fixes

Fix parameter mismatches.

bitset: Cosmetic documentation changes

printeditor: Add desktop file and appdata

All our installed demos have this.

print-editor: Allow opening files

Since it calls itself an editor, it should really
support opening files on the commandline.

printeditor: Cosmetics

Bring this up to our standards for an installable demo,
by touching up the about dialog and menus.

Merge branch 'issue-2900' into 'master'

Nullable this_ argument in gtk_expression_bind()

Closes #2900

See merge request GNOME/gtk!2163

Add pre-condition check on nullable argument

Even if `gtk_expression_watch()` will do the same, we're calling public
API, so we should perform a check at the point of use, to ensure that
warnings are easily debuggable.

Annotate nullable argument

The `this_` argument is passed to `gtk_expression_watch()`, which
accepts it as nullable.

Fixes: #2900

docs: Remove ancient version information

We treat 4.0 as a new era.

multiselection: Update docs

Remove information that is no longer true. GtkMultiSelection
is persistent across sorting changes since 3d8b6f6b799499cca02.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2160

docs: Migration guide additions

builder: Documentation tweaks

Don't use no-longer-existing enums as examples,
and drop the +.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!2159

Update Kazakh translation

Update Kazakh translation

gtk-demo: Drop an unused object

The main ui file was still creating a tree store,
but we've switched everything over to use list models.

builder: Warn about dropped objects

Use GTK_DEBUG=builder-objects to make GtkBuilder warn
if a named object from a ui files doesn't get claimed
by gtk_builder_get_object(). This is useful for finding
dead wood in .ui files.

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #2897 and #2898

See merge request GNOME/gtk!2158

listitem: Correct the docs

gtk-demo: Improve formatting in the color demo

Properly align and format the numbers in the size
dropdown.

text: Support reverse selection

Take ordering of cursor_position and selection_bound
into account when copying text to the clipboard, and
ensure that both orders work the same.

Fixes: #2898

text: Update action enabled state correctly

Take the editable property into account when updating
the emoji.insert action state, and update the action
when it changes.

Fixes: #2897

shortcutcontroller: Add some debug spew

This helps in debugging event routing and keyboard
shortcut issues.

widget: Fix an oversight

g_message comes with a builtin newline, don't add one
to the message.

Merge branch 'matthiasc/for-master' into 'master'

Adwaita: Fix block cursors in spinbuttons

Closes #2871

See merge request GNOME/gtk!2156

Adwaita: Fix block cursors in spinbuttons

Fix block cursors in vertical spinbuttons with Adwaita.

Fixes: #2871

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

Closes #2869

See merge request GNOME/gtk!2155

docs: Add css docs to entries

The various entries were missing the semi-standard
"CSS Nodes" section in their long descriptions. Add
them.

text: Be more selective when selecting on focus-in

We don't want to select on focus-in when the focus
comes from a child. The case where this does harm
is when you activate copy or paste actions from the
context menu. We close the menu before triggering the
action, and if that causes the text in the entry to
be selected, unexpected things happen, since the action
applies to the current selection.

Fixes: #2869

Revert "text: Avoid creating a PangoAttrList we don't need"

This reverts commit 67c26650280ba64ff1c0318be8d848e9563ddb21.

The splicing we do here has the important side-effect
of shifting the preedit attributes to the right position.
Without it, we end up always underlining the first chars
in the entry, regardless where the preedit happens.

text: Handle key events in the target phase

This makes sure that we do actual key input right
in the middle between all the capture and bubble
event controllers, and are not dependent on the
ordering of those controllers.

The bug that triggered this change was that the
shortcut for activation (Enter) was getting triggered
before the key input, causing Ctrl-Shift-u hex
to stop working, since it never received the enter
to commit the sequence.

widget: Add a debug message for consumed key events

Run the application with GTK_DEBUG=keybindings to
get some hints where key events get lost.

text: Give names to event controllers

It helps with debugging.

Merge branch 'wip/carlosg/sequence-state-fixups' into 'master'

Fixups to gesture sequence states

Closes #2895

See merge request GNOME/gtk!2154

gtktext: Don't accept (twice!) all press actions

The gesture should claim the sequence after triggering uncancellable
actions, like pasting, showing a menu or selecting words/lines. A
single first button press initiating a drag does not trigger
anything yet, so it should avoid claiming the sequence.

gtkswitch: Don't be eager in accepting the gesture

The gesture should be accepted whenever it triggers uncancellable
actions in the widget. This means it should be accepted if the
click does result in toggling the switch.

This leaves the pan gesture room to handle dragging the handle.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2895

Merge branch 'matthiasc/for-master' into 'master'

main: Avoid a warning

Closes #2894

See merge request GNOME/gtk!2153

docs: Some tweaks to the list widget overview

Merge branch 'wip/otte/bitset' into 'master'

Improve selection handling API for rubberbanding

See merge request GNOME/gtk!2086

main: Avoid a warning

transient-for relationships only exist between
windows, so check that both candidates are such.

Fixes: #2894

Merge branch 'tintou/dropdown-annotations' into 'master'

dropdown: Annotate the get_selected_item method

See merge request GNOME/gtk!2152

Merge branch 'wip/ricotz/annotations' into 'master'

gtk: Add some g-i annotations to GtkStringList

See merge request GNOME/gtk!2150

dropdown: Annotate the get_selected_item method

We need to specify the type as we already know that it is at least a GObject and in case of no selection, NULL is returned.

listbase: Make rubberbanding a threshold drag

listbase: Only compute the modifiers when releasing the rubberband

... and do the right things:

nothing:    selection = rubberband
ctrl:       selection = selection OR rubberband
shift:      selection = selection AND (NOT rubberband)
ctrl+shift: selection = selection XOR rubberband
            (not sure this one makes sense, but toggling is fun)

listbase: Compute rubberband region on-demand

Instead of storing the active items as we go, compute the affected items
whenever the rubberband changes and in particular when the rubberband
ends.
That way, the rubberband is guaranteed to select a rectangle even
after scrolling very far.

This is achieved by having a get_items_in_rect() vfunc that selects all
the items in the rubberbanded rectangle and returns them as a bitset.

listview: Allocate rubberband at end of size_allocate()

Otherwise the rubberband uses the wrong scroll offsets.

listbase: Flip autoscroll deltas if adjustments are flipped

Fixes autoscroll on RTL languages.

listbase: Allocate gridview items properly on RTL

We need to flip the items.